⚠️ No Changeset found

Latest commit: 1bdcf2d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

This is great, but for me the big one is the sandbox, where we'd want to test that:

• the "app" actually loads
• the html etc tabs works
• changing the code updates the app

Open in StackBlitz

npm i https://pkg.pr.new/@builder.io/qwik@7728

npm i https://pkg.pr.new/@builder.io/qwik-city@7728

npm i https://pkg.pr.new/eslint-plugin-qwik@7728

npm i https://pkg.pr.new/create-qwik@7728

commit: 1bdcf2d

This is great, but for me the big one is the sandbox, where we'd want to test that:

• the "app" actually loads
• the html etc tabs works
• changing the code updates the app

Thanks & acknowledged.

In terms of the sandbox testing the "hello world" section has been done.
I'll keep plugging away at the other sections too, once the whole sandbox has been covered I'll mark this for review.

Things to note re: sandbox testing.

• Auto complete sandbox example does not work on the qwik.dev live site.
• I am testing that the page loads, but cannot test the API response etc..
• Will need to create a new issue later on this.
• raised on Discord to chat about it first.

On the clock page

• Cannot test paragraph on click
• Presumably playwright on clicks on buttons links etc.. but I did try to add a test and it would always fail.
• In short cannot test that the below console.log works.

in the routing.spec.ts file,

• I am testing against the URL change only for now.

The page title is inconsistent to test against, will need to log a new issue against this.
But the bones are setup so once that is resolved it's not a big change.

Current Status:
211 tests - all pass using 1 worker, I did have some timing issues running 8 workers locally.

/>